ARROW-15601: [Docs][Release] Update post release script to move stable docs + keep dev docs#12355
ARROW-15601: [Docs][Release] Update post release script to move stable docs + keep dev docs#12355jorisvandenbossche wants to merge 7 commits intoapache:masterfrom
Conversation
…e docs + keep dev docs
jorisvandenbossche
requested review from
kou and
kszucs
and removed request for
kou
|
One thing I just realized is that the "move current stable to previous version" logic only needs to happen for a major release, and not when doing a bug-fix release (eg 7.0.1). So that logic should maybe be behind a flag (eg based on whether the second "previous_version" parameter was specified?) |
|
How about checking the minor and patch version in the first argument? case "${version}" in
*.0.0)
is_major_release=yes
;;
*)
is_major_release=no
;;
esacIf we need to change the number of arguments depending on context, a human error will occur. |
|
@kou thanks for the suggestion! |
| fi | ||
| git add docs | ||
| git commit -m "[Website] Update documentations for ${version}" | ||
| git clean -d -f -x |
There was a problem hiding this comment.
Not strictly "needed" for the creating the correct PR, but the script does leave a bunch of ignored files after running it locally (eg we don't commit the .doctrees files), and including this ensures it cleans up after itself. I could also limit it to the /docs/ directory (or remove it altogether)
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
|
Sorry for the delay here, this should be ready now I think |
|
Benchmark runs are scheduled for baseline = c4c5c03 and contender = 8963755. 8963755 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
3 participants
This updates the post-release script for the docs with:
/docs/devdocs